Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[server] Add support for proxies #33

Merged
merged 25 commits into from
Sep 19, 2024
Merged

[server] Add support for proxies #33

merged 25 commits into from
Sep 19, 2024

Conversation

grqz
Copy link
Collaborator

@grqz grqz commented Sep 17, 2024 •
edited
Loading

Closes #32
TODO:

  • server-side implementation

@grqz
[plugin] pass proxy to the server
c22d116 
* untested! the committed code may contain potential issues
@grqz grqz marked this pull request as ready for review September 17, 2024 20:17
@Brainicism Brainicism changed the title [server] add support for proxies [server] Add support for proxies Sep 17, 2024
grqz
grqz commented Sep 18, 2024
View reviewed changes
server/src/main.ts Outdated Show resolved Hide resolved
@grqz
remove ellipsis from the features tuple
5928cad 
it's copied from coletdjnz's demo
@grqz grqz added enhancement New feature or request server mainly related to server part of the project labels Sep 18, 2024
grqz
grqz commented Sep 18, 2024
View reviewed changes
server/src/session_manager.ts Outdated Show resolved Hide resolved
@grqz grqz requested a review from Brainicism September 19, 2024 04:09
@grqz grqz merged commit 11b20a4 into master Sep 19, 2024
3 checks passed
@grqz grqz deleted the server/proxy branch September 19, 2024 04:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Brainicism Brainicism Brainicism approved these changes

Assignees
No one assigned
Labels
enhancement New feature or request server mainly related to server part of the project
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Fix readme and proxy question?
3 participants
@grqz @Brainicism @grqx